app: improve code of extract_accels_from_menu()
authorSébastien Wilmet <swilmet@gnome.org>
Sun, 10 Apr 2016 10:41:09 +0000 (12:41 +0200)
committerSébastien Wilmet <swilmet@gnome.org>
Sun, 10 Apr 2016 14:54:07 +0000 (16:54 +0200)
sub_model is clearer than "m". And we don't use the key, so we can pass
NULL instead.

https://bugzilla.gnome.org/show_bug.cgi?id=764846

gtk/gtkapplication.c

index 49c7dd7fdc48d0ce003ac2226da047c9444b06fa..c96cf7a6197fc6134db6b431c025f2312fc82404 100644 (file)
@@ -829,19 +829,19 @@ extract_accels_from_menu (GMenuModel     *model,
                           GtkApplication *app)
 {
   gint i;
-  GMenuLinkIter *iter;
-  const gchar *key;
-  GMenuModel *m;
 
   for (i = 0; i < g_menu_model_get_n_items (model); i++)
     {
+      GMenuLinkIter *iter;
+      GMenuModel *sub_model;
+
       extract_accel_from_menu_item (model, i, app);
 
       iter = g_menu_model_iterate_item_links (model, i);
-      while (g_menu_link_iter_get_next (iter, &key, &m))
+      while (g_menu_link_iter_get_next (iter, NULL, &sub_model))
         {
-          extract_accels_from_menu (m, app);
-          g_object_unref (m);
+          extract_accels_from_menu (sub_model, app);
+          g_object_unref (sub_model);
         }
       g_object_unref (iter);
     }